Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reduce request memory to 11MB. #34

Merged
merged 1 commit into from
Dec 19, 2023
Merged

Reduce request memory to 11MB. #34

merged 1 commit into from
Dec 19, 2023

Conversation

kasperg
Copy link
Contributor

@kasperg kasperg commented Dec 19, 2023

This is based on a sample of 10.000 requests after launching on Platform.sh:

platform log --lines 10000 php.access | awk '{print $6}' | sort -n | uniq -c Reading log file [email protected]:/var/log/php.access.log 9598 2048
401 4096
1 6144

We might even be able to go lower but 10MB is the minimum value on Platform.sh but setting this value leads to weird redirection errors between FPM-FPM and NGINX so go one up with 11MB.
https://docs.platform.sh/languages/php/fpm.html#2-adjust-the-maximum-number-of-php-fpm-workers

This is based on a sample of 10.000 requests after launching on 
Platform.sh:

platform log --lines 10000 php.access | awk '{print $6}' | sort -n | uniq -c
Reading log file [email protected]:/var/log/php.access.log
9598 2048
 401 4096
   1 6144

We might even be able to go lower but 10MB is the minimum value on
Platform.sh but setting this value leads to weird redirection errors
between FPM-FPM and NGINX so go one up with 11MB.
https://docs.platform.sh/languages/php/fpm.html#2-adjust-the-maximum-number-of-php-fpm-workers
@kasperg kasperg merged commit e6883ab into master Dec 19, 2023
9 checks passed
@kasperg kasperg deleted the request-memory-11MB branch December 19, 2023 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant